Update from update/networkservicemesh/sdk #154
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…k@main networkservicemesh/sdk# networkservicemesh/sdk PR link: https://github.com/networkservicemesh/sdk/pull/ networkservicemesh/sdk commit message: commit 19c7a102f2a7bb9914763ec77304a69ce7790f0e Author: Uzlov Danil <36223296+d-uzlov@users.noreply.github.com> Date: Thu Apr 22 12:47:30 2021 +0700 Add Client Monitor to the Heal Client (#810) * Fix selector handling in monitorServer.MonitorConnections Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * Move event monitoring from heal server to heal client Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * Fix heal not working in sandbox tests Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * Fix pending issues and clean code in heal package Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * Fix heal client not working reliably on first request Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * Fix issues, discussed in PR comments Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * Fix heal unit test panic Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * Fix imports layout Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * Fix heal client using wrong id when listening to connection updates Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * Stop heal client from potentially sending repeating heal requests Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * Use updated connection path on heal Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * Move connection path restoring from heal server to heal client Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * In heal client use mutex per connection instead of one executor Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * In heal server use mutex per connection instead of one executor Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * Fix linter warning Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * Add comments to "heal" package Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * Fix heal client not saving endpoint name on connection update events Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * Make heal client wait for confirmation in Request if connection was broken Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * extract initialization method in heal client Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * use sync.Cond instead of channel to confirm connection in heal client Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * use separate functions for "heal" and "restore connection" Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * use better error message in heal client Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * fix file formatting Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * refactor heal server code Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * use single type for both heal requests Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * combine types in healClient.listenToConnectionChanges() Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * remove event-check in heal client, remove incorrect heal test Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * remove complex state from conn info in heal client Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * use mutex instead of sync.Cond in heal client Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * fix linter warning Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * fix heal client not removing unsuccessful connection Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * fix heal server not using clock package during healing Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot
force-pushed
the
update/networkservicemesh/sdk
branch
from
18f2943 to
3288936
Compare
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
commit 18f2943 Author: NSMBot nsmbot@networkservicmesh.io Date: Thu Apr 22 05:48:08 2021 +0000 Update go.mod and go.sum to latest version from networkservicemesh/sdk@main networkservicemesh/sdk# networkservicemesh/sdk PR link: https://github.com/networkservicemesh/sdk/pull/ networkservicemesh/sdk commit message: commit 19c7a102f2a7bb9914763ec77304a69ce7790f0e Author: Uzlov Danil 36223296+d-uzlov@users.noreply.github.com Date: Thu Apr 22 12:47:30 2021 +0700 Add Client Monitor to the Heal Client (#810) Dockerfile LICENSE README.md go.mod go.sum internal main.go staticcheck.conf suite_healthcheck_test.go suite_setup_test.go suite_test.go suite_util_test.go Fix selector handling in monitorServer.MonitorConnections Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru Dockerfile LICENSE README.md go.mod go.sum internal main.go staticcheck.conf suite_healthcheck_test.go suite_setup_test.go suite_test.go suite_util_test.go Move event monitoring from heal server to heal client Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru Dockerfile LICENSE README.md go.mod go.sum internal main.go staticcheck.conf suite_healthcheck_test.go suite_setup_test.go suite_test.go suite_util_test.go Fix heal not working in sandbox tests Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru Dockerfile LICENSE README.md go.mod go.sum internal main.go staticcheck.conf suite_healthcheck_test.go suite_setup_test.go suite_test.go suite_util_test.go Fix pending issues and clean code in heal package Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru Dockerfile LICENSE README.md go.mod go.sum internal main.go staticcheck.conf suite_healthcheck_test.go suite_setup_test.go suite_test.go suite_util_test.go Fix heal client not working reliably on first request Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru Dockerfile LICENSE README.md go.mod go.sum internal main.go staticcheck.conf suite_healthcheck_test.go suite_setup_test.go suite_test.go suite_util_test.go Fix issues, discussed in PR comments Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru Dockerfile LICENSE README.md go.mod go.sum internal main.go staticcheck.conf suite_healthcheck_test.go suite_setup_test.go suite_test.go suite_util_test.go Fix heal unit test panic Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru Dockerfile LICENSE README.md go.mod go.sum internal main.go staticcheck.conf suite_healthcheck_test.go suite_setup_test.go suite_test.go suite_util_test.go Fix imports layout Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru Dockerfile LICENSE README.md go.mod go.sum internal main.go staticcheck.conf suite_healthcheck_test.go suite_setup_test.go suite_test.go suite_util_test.go Fix heal client using wrong id when listening to connection updates Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru Dockerfile LICENSE README.md go.mod go.sum internal main.go staticcheck.conf suite_healthcheck_test.go suite_setup_test.go suite_test.go suite_util_test.go Stop heal client from potentially sending repeating heal requests Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru Dockerfile LICENSE README.md go.mod go.sum internal main.go staticcheck.conf suite_healthcheck_test.go suite_setup_test.go suite_test.go suite_util_test.go Use updated connection path on heal Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru Dockerfile LICENSE README.md go.mod go.sum internal main.go staticcheck.conf suite_healthcheck_test.go suite_setup_test.go suite_test.go suite_util_test.go Move connection path restoring from heal server to heal client Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru Dockerfile LICENSE README.md go.mod go.sum internal main.go staticcheck.conf suite_healthcheck_test.go suite_setup_test.go suite_test.go suite_util_test.go In heal client use mutex per connection instead of one executor Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru Dockerfile LICENSE README.md go.mod go.sum internal main.go staticcheck.conf suite_healthcheck_test.go suite_setup_test.go suite_test.go suite_util_test.go In heal server use mutex per connection instead of one executor Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru Dockerfile LICENSE README.md go.mod go.sum internal main.go staticcheck.conf suite_healthcheck_test.go suite_setup_test.go suite_test.go suite_util_test.go Fix linter warning Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru Dockerfile LICENSE README.md go.mod go.sum internal main.go staticcheck.conf suite_healthcheck_test.go suite_setup_test.go suite_test.go suite_util_test.go Add comments to "heal" package Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru Dockerfile LICENSE README.md go.mod go.sum internal main.go staticcheck.conf suite_healthcheck_test.go suite_setup_test.go suite_test.go suite_util_test.go Fix heal client not saving endpoint name on connection update events Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru Dockerfile LICENSE README.md go.mod go.sum internal main.go staticcheck.conf suite_healthcheck_test.go suite_setup_test.go suite_test.go suite_util_test.go Make heal client wait for confirmation in Request if connection was broken Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru Dockerfile LICENSE README.md go.mod go.sum internal main.go staticcheck.conf suite_healthcheck_test.go suite_setup_test.go suite_test.go suite_util_test.go extract initialization method in heal client Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru Dockerfile LICENSE README.md go.mod go.sum internal main.go staticcheck.conf suite_healthcheck_test.go suite_setup_test.go suite_test.go suite_util_test.go use sync.Cond instead of channel to confirm connection in heal client Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru Dockerfile LICENSE README.md go.mod go.sum internal main.go staticcheck.conf suite_healthcheck_test.go suite_setup_test.go suite_test.go suite_util_test.go use separate functions for "heal" and "restore connection" Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru Dockerfile LICENSE README.md go.mod go.sum internal main.go staticcheck.conf suite_healthcheck_test.go suite_setup_test.go suite_test.go suite_util_test.go use better error message in heal client Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru Dockerfile LICENSE README.md go.mod go.sum internal main.go staticcheck.conf suite_healthcheck_test.go suite_setup_test.go suite_test.go suite_util_test.go fix file formatting Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru Dockerfile LICENSE README.md go.mod go.sum internal main.go staticcheck.conf suite_healthcheck_test.go suite_setup_test.go suite_test.go suite_util_test.go refactor heal server code Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru Dockerfile LICENSE README.md go.mod go.sum internal main.go staticcheck.conf suite_healthcheck_test.go suite_setup_test.go suite_test.go suite_util_test.go use single type for both heal requests Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru Dockerfile LICENSE README.md go.mod go.sum internal main.go staticcheck.conf suite_healthcheck_test.go suite_setup_test.go suite_test.go suite_util_test.go combine types in healClient.listenToConnectionChanges() Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru Dockerfile LICENSE README.md go.mod go.sum internal main.go staticcheck.conf suite_healthcheck_test.go suite_setup_test.go suite_test.go suite_util_test.go remove event-check in heal client, remove incorrect heal test Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru Dockerfile LICENSE README.md go.mod go.sum internal main.go staticcheck.conf suite_healthcheck_test.go suite_setup_test.go suite_test.go suite_util_test.go remove complex state from conn info in heal client Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru Dockerfile LICENSE README.md go.mod go.sum internal main.go staticcheck.conf suite_healthcheck_test.go suite_setup_test.go suite_test.go suite_util_test.go use mutex instead of sync.Cond in heal client Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru Dockerfile LICENSE README.md go.mod go.sum internal main.go staticcheck.conf suite_healthcheck_test.go suite_setup_test.go suite_test.go suite_util_test.go fix linter warning Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru Dockerfile LICENSE README.md go.mod go.sum internal main.go staticcheck.conf suite_healthcheck_test.go suite_setup_test.go suite_test.go suite_util_test.go fix heal client not removing unsuccessful connection Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru Dockerfile LICENSE README.md go.mod go.sum internal main.go staticcheck.conf suite_healthcheck_test.go suite_setup_test.go suite_test.go suite_util_test.go fix heal server not using clock package during healing Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru Signed-off-by: NSMBot nsmbot@networkservicmesh.io